Skip to content

RevitPackageDocumentation: Доработка функционала конфигурации#474

Open
DivinN wants to merge 5 commits into
masterfrom
DivinN/RevitPackageDocumentationFix
Open

RevitPackageDocumentation: Доработка функционала конфигурации#474
DivinN wants to merge 5 commits into
masterfrom
DivinN/RevitPackageDocumentationFix

Conversation

@DivinN

@DivinN DivinN commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Внесены изменения:

  • Добавлена возможность снимать шаблон вида после создания вида
  • Добавлены комментарии для параметров комплекта
  • Решена проблема с вылетом приложения при нажатии Esc при выборе элемента в параметр комплекта
  • Вызов метода валидации экземпляра объекта перенесен на класс валидации
  • Изменен способ назначения Номера вида у видов в плане, сечения и фрагмента - применены уникальные строки для типов компонентов (уникальные строки не пустые. Они содержат регламентированные проектировщиками невидимые символы, которые они используются в качестве префикса в Номере вида, чтобы можно было назначать одинаковый номер вида)
image

@DivinN DivinN requested a review from vlastroG July 10, 2026 10:21
@DivinN DivinN self-assigned this Jul 10, 2026
Comment on lines +16 to +17
private readonly string _uniqueViewportNumberKey = "‪";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

здесь добавь комментарий почему тут скрытый символ и какой он по unicod

Comment on lines +17 to +18
private readonly string _uniqueViewportNumberKey = "‏";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

коммент про скрытый символ

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если это один и тот же символ, что и в CalloutViewVM, то лучше сделать одну публичную константу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants